Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

public/add-fair-usage-policy-link #583

Merged
merged 4 commits into from
Oct 22, 2024

Conversation

jszuminski
Copy link
Contributor

@jszuminski jszuminski commented Oct 7, 2024

The only change in this pull request is adding the fair usage policy link. The other changes are related to prettier of the index.html file.


Internal note:

The updated page looks exactly the same as it used to. Only an additional sentence has been added to the description below the <h1>, including now the fair usage policy link (same as readme).

Screenshot 2024-10-08 at 06 21 10

public/index.html Fixed Show resolved Hide resolved
public/index.html Fixed Show fixed Hide fixed
public/index.html Fixed Show fixed Hide fixed
@jszuminski jszuminski self-assigned this Oct 8, 2024
@jszuminski
Copy link
Contributor Author

When it comes to the other part of the task (the User-Agent and Referer headers), these headers are automatically included in the request sent from the client-side, from the browser.

Thus, there's no need (and even no possibility) to modify them. So the ajax() in this public/index.html remains correct.

Copy link
Contributor

@cvasseng cvasseng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you take a look at the integrity warnings from the security advisor and update where possible?

@jszuminski jszuminski merged commit 0c34ece into master Oct 22, 2024
4 checks passed
@jszuminski jszuminski deleted the public/add-fair-usage-policy-link branch October 22, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants